Skip to content

Conversation

joereg4
Copy link

@joereg4 joereg4 commented Oct 10, 2025

Fixes #1622

Description

Adds comprehensive documentation for the Nix binary cache system used in the Supabase PostgreSQL project.

Changes

  • Created nix/docs/nix-cache.md with comprehensive cache documentation
  • Updated nix/docs/README.md to include cache docs in navigation
  • Updated nix/mkdocs.yml to add cache docs to MkDocs navigation
  • All tests pass (nix flake check -L)

Closes #1622"

@joereg4 joereg4 requested review from a team as code owners October 10, 2025 23:12
@joereg4
Copy link
Author

joereg4 commented Oct 10, 2025

I noticed the CI failed due to missing AWS credentials. Are contributors supposed to receive credentials or should those steps skip in forks / PRs?

@joereg4
Copy link
Author

joereg4 commented Oct 11, 2025

Hello @samrose,

I'm a new contributor working on a PR for issue #1622 (Nix cache documentation). I've encountered the AWS credentials CI failure on my fork's PR and wanted to reach out for some guidance.

I've observed that the run-testinfra jobs fails with DEV_AWS_ROLE credential errors on fork PRs. This appears to be GitHub's security model preventing forks from accessing repository secrets. Looking through PRs and I do see this is impacting others as well.

All my local and other CI tests pass and the code changes are documentation-focused (new nix-cache.md file).

Since you are the #1 contributor, what's the best way forward for contributors facing this CI limitation? Should I wait for a maintainer to manually review/approve the AWS tests, or is there something else I should be doing?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Nix cache is not documented

1 participant